Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typos, grammar and spacing #375

Merged
merged 8 commits into from
Jul 31, 2023
Merged

Fix typos, grammar and spacing #375

merged 8 commits into from
Jul 31, 2023

Conversation

galoget
Copy link
Contributor

@galoget galoget commented Jul 25, 2023

Fixed some typos, grammar on some sentences and standardized spacing in comments.

Fix typo and spacing.
Fix typos, spacing and added comments.
Fix typos and spacing
Standardize spacing in comments
Standardize spacing in comments
Standardize spacing in comments.
Standardize spacing in comments.
Fix typos, grammar and misspelled words.
@github-actions
Copy link

The provided patches do not seem to contain any security issues, vulnerabilities, backdoors, potentially malicious or suspicious code, leaks of sensitive information, code bugs, or optimization issues.

However, there are a few minor code smells that could be improved for better readability and maintainability:

  1. linPEAS/builder/linpeas_builder.py:

    • Line 6, 9, 20, 30: Comments should be separated by a space from the code for better readability.
  2. linPEAS/builder/linpeas_parts/2_container.sh:

    • Line 16, 257, 279: Comments should be separated by a space from the code for better readability.
  3. linPEAS/builder/linpeas_parts/3_cloud.sh:

    • Line 150, 236, 270, 334, 346, 371, 398, 436: Comments should be separated by a space from the code for better readability.
    • Line 436: There's a typo in the comment, "metdata" should be "metadata".

These are not critical issues but following best practices for code comments can improve the readability and maintainability of your code.

@github-actions
Copy link

The provided PR patches are mostly related to comment changes and minor code adjustments. Here are the details:

  1. Security issues: No security issues were found in the provided patches.

  2. Vulnerabilities: No vulnerabilities were found in the provided patches.

  3. Backdoors: No backdoors were found in the provided patches.

  4. Potentially malicious or suspicious code: No potentially malicious or suspicious code was found in the provided patches.

  5. Leaks of sensitive information: No leaks of sensitive information were found in the provided patches.

  6. Code bugs: No code bugs were found in the provided patches.

  7. Optimization issues: No optimization issues were found in the provided patches.

  8. Code smells:

    • File: linPEAS/builder/linpeas_parts/7_software_information.sh
    • Lines: 144, 239, 352, 533, 546
    • Issue: The comments in these lines are not clear and concise. They should be rewritten for better understanding and maintainability.

Overall, the code seems to be well-written with no major issues detected in the provided patches. However, the comments could be improved for better readability and understanding.

@carlospolop carlospolop merged commit 9861259 into peass-ng:master Jul 31, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants